Skip to content

✨ Move doc generation code away from cmd into lrcore, alongside tests#6030

Merged
preslavgerchev merged 1 commit intomainfrom
preslav/shuffle-lr-code
Oct 10, 2025
Merged

✨ Move doc generation code away from cmd into lrcore, alongside tests#6030
preslavgerchev merged 1 commit intomainfrom
preslav/shuffle-lr-code

Conversation

@preslavgerchev
Copy link
Copy Markdown
Contributor

@preslavgerchev preslavgerchev commented Oct 9, 2025

  • Move doc generation code into lrcore
  • Move tests from cmd into lrcore
  • Add GenerateDocs fn to the LR struct.

The cmd package now has very little logic and most of this is in lrcore, allowing us to easily test this

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Oct 9, 2025

Test Results

4 511 tests  +1   4 507 ✅ +1   2m 13s ⏱️ -1s
  409 suites ±0       4 💤 ±0 
   30 files   ±0       0 ❌ ±0 

Results for commit 85a4984. ± Comparison against base commit ee64f7b.

This pull request removes 1 and adds 2 tests. Note that renamed tests count towards both.
go.mondoo.com/cnquery/v12/providers-sdk/v1/mqlr/cmd ‑ TestPlatformMapping
go.mondoo.com/cnquery/v12/providers-sdk/v1/mqlr/lrcore ‑ TestGenerateDocs
go.mondoo.com/cnquery/v12/providers-sdk/v1/mqlr/lrcore ‑ TestPlatformMapping

♻️ This comment has been updated with latest results.

@preslavgerchev preslavgerchev force-pushed the preslav/shuffle-lr-code branch 4 times, most recently from 0080b5b to 4c09562 Compare October 9, 2025 11:25
Copy link
Copy Markdown
Member

@chris-rock chris-rock left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you @preslavgerchev

@preslavgerchev preslavgerchev force-pushed the preslav/shuffle-lr-code branch 3 times, most recently from a0027d3 to 5e69bc7 Compare October 10, 2025 19:35
…. the LR ast now has a GenerateDocs function.

Signed-off-by: Preslav <preslav@mondoo.com>
@preslavgerchev preslavgerchev force-pushed the preslav/shuffle-lr-code branch from 5e69bc7 to 85a4984 Compare October 10, 2025 19:37
@preslavgerchev preslavgerchev merged commit e0cee31 into main Oct 10, 2025
24 of 25 checks passed
@preslavgerchev preslavgerchev deleted the preslav/shuffle-lr-code branch October 10, 2025 20:02
@github-actions github-actions bot locked and limited conversation to collaborators Oct 10, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants